test this please.

QA tests have started for PR 2407 at commit 0d69eb4.

• This patch merges cleanly.

QA tests have finished for PR 2407 at commit 0d69eb4.

• This patch passes unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

We may need to coordinate this with #2241, but otherwise I agree this is much cleaner than before. Thanks for looking into this!

Can you maybe change the title to something like ... Use GenericUDFUtils.ConversionHelper for Simple UDF type conversions?

Thanks for referencing pr 2241 here, while I think this should be reviewed first, and will make 2241 simpler, as well as getting date type support into catalyst earlier.

I add a test case to this file, the current master will fail the case because we missed Short in the primitiveTypes here. With this new solution, we can get rid of maintaining of primitiveTypes, and eliminate potential bugs here.

QA tests have started for PR 2407 at commit 15762d2.

• This patch merges cleanly.

QA tests have finished for PR 2407 at commit 15762d2.

• This patch passes unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

+1, LGTM. I think this can go first, as it's simpler and cleaner (maybe fixed some unknown bugs), besides, it blocks the #2344.

Thanks! I've merged this to master.